Skip to content

Revert "Honour isrecursive above listall"#6126

Merged
nvazquez merged 1 commit into
mainfrom
revert-6045-fix-recursive
Mar 17, 2022
Merged

Revert "Honour isrecursive above listall"#6126
nvazquez merged 1 commit into
mainfrom
revert-6045-fix-recursive

Conversation

@nvazquez

Copy link
Copy Markdown
Contributor

Reverts #6045

Have found some regressions after merging this PR, reverting so we can fix properly

@weizhouapache weizhouapache left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1 on revert

@nvazquez

Copy link
Copy Markdown
Contributor Author

@blueorangutan package

@blueorangutan

Copy link
Copy Markdown

@nvazquez a Jenkins job has been kicked to build packages. It will be bundled with KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

@blueorangutan

Copy link
Copy Markdown

Packaging result: ✔️ el7 ✔️ el8 ✔️ debian ✔️ suse15. SL-JID 2905

@nvazquez

Copy link
Copy Markdown
Contributor Author

@blueorangutan test

@blueorangutan

Copy link
Copy Markdown

@nvazquez a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

@nvazquez nvazquez mentioned this pull request Mar 16, 2022
10 tasks
@blueorangutan

Copy link
Copy Markdown

Trillian test result (tid-3639)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 33010 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr6126-t3639-kvm-centos7.zip
Smoke tests completed. 91 look OK, 1 have errors
Only failed tests results shown below:

Test Result Time (s) Test File
test_hostha_enable_ha_when_host_disabled Error 1.73 test_hostha_kvm.py
test_hostha_enable_ha_when_host_in_maintenance Error 303.78 test_hostha_kvm.py

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

No open projects
Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants